Review requested:

• @nodejs/modules

@guybedford By the way, in PACKAGE_RESOLVE method step 11.1, i thought packageURL should be:

the URL resolution of "node_modules/" concatenated with packageName, relative to parentURL.

Instead of:

11. While parentURL is not the file system root,
    1. Let packageURL be the URL resolution of "node_modules/" concatenated with packageSpecifier, relative to parentURL.

Is this another typo?

Take code from package.md as an example:

// ./node_modules/es-module-package/package.json
{
  "exports": {
    "./features/*": "./src/features/*.js"
  },
  "imports": {
    "#internal/*": "./src/internal/*.js"
  }
}

import featureY from 'es-module-package/features/y/y';
// Loads ./node_modules/es-module-package/src/features/y/y.js

If concatenated "node_modules/" with packageSpecifier, the packageURL will initially be "./node_modules/es-module-package/src/features/y/y", though according to the loop, it will finally locate packageURL to "./node_modules/es-module-package", but it's a waste of time doing meaningless loop for "src/features/y" path.

@yu521088 well spotted, yes that is another typo!

Landed in f458f1b